<html>
<head><meta charset="utf-8"><title>Protected branches? · clippy · Zulip Chat Archive</title></head>
<h2>Stream: <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/index.html">clippy</a></h2>
<h3>Topic: <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/Protected.20branches.3F.html">Protected branches?</a></h3>

<hr>

<base href="https://rust-lang.zulipchat.com">

<head><link href="https://rust-lang.github.io/zulip_archive/style.css" rel="stylesheet"></head>

<a name="212121671"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/Protected%20branches%3F/near/212121671" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Eduardo Broto <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/Protected.20branches.3F.html#212121671">(Oct 02 2020 at 20:01)</a>:</h4>
<p>So, every time I'm reviewing a pull request I see that green merge button looking at me like "yep, you're gonna click me someday". If I'm not mistaken, rust-lang/rust uses protected branches, should we do the same? AFAIK we always use bors anyway.</p>



<a name="212122175"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/Protected%20branches%3F/near/212122175" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> flip1995 <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/Protected.20branches.3F.html#212122175">(Oct 02 2020 at 20:06)</a>:</h4>
<p>The master branch is already protected. We could add a rule, that only bors can push to the master branch, though (<span class="user-mention" data-user-id="132040">@Manish Goregaokar</span>). Every other branch we can't protect, since force pushes are sometimes required.</p>



<a name="212122279"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/Protected%20branches%3F/near/212122279" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Manish Goregaokar <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/Protected.20branches.3F.html#212122279">(Oct 02 2020 at 20:06)</a>:</h4>
<p>i can try to do that</p>



<a name="212122295"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/Protected%20branches%3F/near/212122295" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Manish Goregaokar <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/Protected.20branches.3F.html#212122295">(Oct 02 2020 at 20:06)</a>:</h4>
<p>admins can force push though</p>



<a name="212123001"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/Protected%20branches%3F/near/212123001" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Eduardo Broto <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/Protected.20branches.3F.html#212123001">(Oct 02 2020 at 20:12)</a>:</h4>
<blockquote>
<p>Every other branch we can't protect, since force pushes are sometimes required</p>
</blockquote>
<p>Yeah the idea would be to avoid accidentally merging a PR to master, if possible. I think it can be reverted right away, but better prevent that lament <span aria-label="smile" class="emoji emoji-1f642" role="img" title="smile">:smile:</span></p>



<hr><p>Last updated: Aug 07 2021 at 22:04 UTC</p>
</html>